feat: add demo shutdown notice banner across all pages#229
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a prominent, site-wide notice that Torrust public demo services are shut down, and redirects several demo URLs to the tracking issue (#228) to avoid sending users to broken endpoints.
Changes:
- Added a full-width shutdown banner above the header in the root layout.
- Repointed “demo/live demo” URLs in key homepage components and comparison-table data to issue #228.
- Minor homepage styling change (currently introduces a duplicated selector / likely unintended styling regression).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/routes/+layout.svelte | Adds the global demo shutdown banner and its styling. |
| src/routes/(home)/+page.svelte | Updates homepage styling (currently duplicates .container selector, likely meant to style h2). |
| src/lib/constants/constants.ts | Redirects index/tracker table demo URLs to issue #228. |
| src/lib/components/singletons/TorrustIndexPost.svelte | Redirects the “demo site” link to issue #228 (copy still claims the demo is running). |
| src/lib/components/organisms/Hero.svelte | Redirects HeroCard “Live demo” targets to issue #228 (button label remains “Live demo”). |
Comments suppressed due to low confidence (1)
src/routes/(home)/+page.svelte:63
- The styles define .container twice; the second block looks like it was intended for the h2 but now applies font-size/font-weight to the whole container, likely affecting all nested text and overriding earlier container padding.
.container {
text-align: center;
color: rgba(245, 245, 245, 0.96);
padding-top: 4rem;
font-size: 1.8rem;
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Fixed by adding |
|
Updated the following blog posts to replace or note dead demo links:
Historical code examples (like |
|
Fixed. Updated the text in |
|
Fixed. Updated |
- Add rel="noopener noreferrer" to banner link - Update TorrustIndexPost text to reflect demo shutdown - Change HeroCard button to show "Unavailable" when demo is down - Fix duplicated .container CSS selector in homepage - Replace broken demo links in blog posts with issue reference
|
ACK 26c69f7 |
Closes #228
Adds a prominent full-width banner at the very top of every page (above the header) notifying visitors that all Torrust public demo services are currently shut down.
Changes
New banner in layout (
src/routes/+layout.svelte) — Dark-themed banner with red left accent bar and red border, positioned above the navigation header so it's visible on every page.Removed old subtle notice (
src/lib/components/organisms/Hero.svelte) — The tiny.demo-noticeparagraph from the hero section has been removed (was hard to read and easy to miss).Demo links redirected to issue Add banner on homepage informing that all Torrust public demos are currently down #228 — All "Live demo" buttons and demo URLs now point to the issue instead of broken demo endpoints:
HeroCardlive demo buttons (Index & Tracker)TorrustIndexPost.sveltedemo site linkconstants.tsdemo fields in both index and tracker comparison tablesRevert
See the comment on issue #228 for a full revert checklist to undo these changes when the demos are back online.